Skip to content

feat(chart): add links to data point tooltips - #1438

Open
lovasoa wants to merge 8 commits into
mainfrom
feature/chart-point-links
Open

feat(chart): add links to data point tooltips#1438
lovasoa wants to merge 8 commits into
mainfrom
feature/chart-point-links

Conversation

@lovasoa

@lovasoa lovasoa commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

Closes #945.

Adds an optional row-level link parameter to the chart component. When a chart data row has a link, the chart tooltip includes an accessible Open link anchor pointing to that URL.

Implementation

  • Carries link through the chart template's point data without changing the existing color/z positions.
  • Preserves links on aligned series points.
  • Uses a small DOM-based tooltip renderer only for charts that contain links; charts without links keep ApexCharts' existing tooltip behavior.
  • Handles range bars and other point-based charts, including pie-series data stored by ApexCharts as scalar values.
  • Documents the parameter and adds a timeline example.

Tests

  • npm test
  • cargo test
  • cd tests/end-to-end && npm run test -- --grep "data point link"

Comment thread examples/official-site/sqlpage/migrations/01_documentation.sql
Comment thread sqlpage/apexcharts.js Outdated
Comment thread sqlpage/apexcharts.js
Comment thread tests/end-to-end/fixtures/chart/test.ts Outdated
Comment thread sqlpage/apexcharts.js Outdated
Comment thread tests/components/chart_point_serialization.sql Outdated
@lovasoa

lovasoa commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

Thank you very much, @81reap ! Switching this back to draft while we iterate

@lovasoa
lovasoa marked this pull request as draft September 9, 2026 08:36
@lovasoa

lovasoa commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

Visual verification after the accessible tooltip-link styling update:

rangeBar

line

bar

scatter

@81reap

81reap commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

sorry, I didn't realize this was still in draft. happy to do code reviews whenever tho!

@lovasoa
lovasoa marked this pull request as ready for review September 10, 2026 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chart component : link parameter

2 participants